Skip to content

Conversation

ddebrunner
Copy link
Contributor

No description provided.

@ddebrunner ddebrunner changed the title Persist Persisted documents snippet. Sep 5, 2024
@ddebrunner ddebrunner marked this pull request as ready for review September 5, 2024 22:40
Copy link
Contributor

@bobbiejc bobbiejc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

authTypes,
getTestDescription,
} = require("../../../tests/gqltest.js");

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any way to define the REPO_ROOT like we have done in our Makefiles? this ../../../.. nonsense always catches me. somehow I could do this once at the top of the file more easily -- I guess in this case it's only used once...

Co-authored-by: Bobbie Cochrane <[email protected]>
@ddebrunner ddebrunner merged commit 600a350 into main Sep 6, 2024
1 check passed
@ddebrunner ddebrunner deleted the persist branch September 6, 2024 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants